Codecov Report

Merging #21 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #21   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines          18     13    -5     
=====================================
- Hits           18     13    -5

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3b21f6e...38f89ba. Read the comment docs.

@alanshaw @vmx what's the state of this?

@hacdias ipfs/js-ipfs#1670 (comment)

@vmx gotcha! Then, perhaps, shouldn't we close this PRs on ipld side? Since there are major refactorings coming?

I first thought "yes, let's close it". But then after a second thought I might leave it open. I personally look at open issues/PRs more than on closed one. There's certainly a discoverability issue here, that the refactoring is coming. So my hope would be that anyone trying to hop in to help with the async/await endeavour, to find those already open PRs and won't work on it.

What do you think @hacdias?

@vmx I agree with the (in)visibility of the issues/PRs when they're closed/open. Although I also find it distracting and we always have ipfs/js-ipfs#1670 to keep track of all of the async await changes.

Superseded by #32.